Repository navigation
SK-3262: bump skyvault to 2.2.0 for public release (Detect V2) - #294
Merged
saileshwar-skyflow merged 2 commits intoNov 13, 2026
Merged
Conversation
Detect V2 landed on main (shared common/, re-exported from skyvault) after the 2.1.3 release, which did not include it. Bump the skyvault SDK (dist skyflow) to 2.2.0 -- a minor release for the new detect feature. Both setup.py and _version.py are patched by bump_version.sh and must match. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
✅ Gitleaks Findings: No secrets detected. Safe to proceed! |
skyvault/.coveragerc did not omit setup.py, so a version-only release PR (bumping setup.py current_version + _version.py SDK_VERSION) measured setup.py's changed line as uncovered -> codecov/patch 50% < 85% target. flowvault/.coveragerc already omits setup.py, which is why its identical bump PR passed. Match it so skyvault measures coverage the same way. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
✅ Gitleaks Findings: No secrets detected. Safe to proceed! |
skyflow-bharti
approved these changes
Nov 13, 2026
skyflow-himanshupal
approved these changes
Nov 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Bumps the skyvault SDK (dist
skyflow) from2.1.3→2.2.0for the public PyPI release that includes Detect V2.skyvault/setup.py→current_version = '2.2.0'skyvault/skyflow/utils/_version.py→SDK_VERSION = '2.2.0'Both files are patched by
bump_version.shand must stay in lockstep.Why
Detect V2 (deidentify/reidentify string+file, getRun, checkGuardrails) landed on
mainvia #292 (sharedcommon/, re-exported fromskyvault/skyflow/vault/detect) after the2.1.3release, which did not ship detect.2.2.0is a minor bump for the new feature.Verified (already on main, no changes needed here)
DeidentifyFileDataFormat/ReidentifyFileDataFormat), no leftoverFileDataFormat.DeidentifyFileDataFormat, 9ReidentifyFileDataFormat).2.2.0-equivalent build (SK-3262 branch in sdk-integration-tests).Release steps after merge
main.2.2.0targetingmain.release.ymlpublishesskyflow==2.2.0to PyPI.Scope
Version-only change in this PR — no functional diff.
🤖 Generated with Claude Code